feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/component/FilterChip.kt (1)

47-57: 중요: .then 블록에서 파라미터 modifier 재사용으로 수정자 중복 적용 가능성

.then(if (isSelected) { modifier } else { modifier.border(...) })는 파라미터로 받은 modifier를 체인 끝에 다시 붙입니다. 호출 측에서 modifier에 padding/size/semantics 등이 포함되어 있으면 두 번 적용될 수 있습니다. 의도는 빈 Modifier 또는 border만 추가하는 것이므로 아래처럼 교체 바랍니다.

예시:

.then(
    if (isSelected) {
        Modifier
    } else {
        Modifier.border(
            width = 1.dp,
            color = ReedTheme.colors.borderPrimary,
            shape = RoundedCornerShape(ReedTheme.radius.full),
        )
    },
)

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt (1)

62-83: 빈 목록일 때 런타임 크래시 가능 — emotionTags[0] 접근 가드 필요

emotionTags가 비어있으면 인덱싱(0)에서 예외가 발생합니다. 실제 Preview에서도 빈 리스트를 넘기고 있어 재현 가능성이 높습니다. 안전하게 처리하도록 가드하거나 기본값을 두는 편이 좋습니다.

예시 수정(diff는 해당 블록에 맞춰 최소 변경으로 제안):

         Row(
             modifier = Modifier.fillMaxWidth(),
             verticalAlignment = Alignment.CenterVertically,
         ) {
+            val emotion = emotionTags.firstOrNull()
             Image(
-                painter = painterResource(getEmotionImageResourceByDisplayName(emotionTags[0])),
+                painter = painterResource(getEmotionImageResourceByDisplayName(emotion ?: "따뜻함")),
                 contentDescription = "Emotion Graphic",
                 modifier = Modifier
                     .size(40.dp)
-                    .clip(CircleShape)
-                    .background(ReedTheme.colors.basePrimary),
+                    .clip(CircleShape)
+                    .background(ReedTheme.colors.basePrimary, CircleShape),
             )
             Spacer(modifier = Modifier.width(ReedTheme.spacing.spacing2))
             Column {
-                Text(
-                    text = "#${emotionTags[0]}",
-                    color = ReedTheme.colors.contentBrand,
-                    style = ReedTheme.typography.label1SemiBold,
-                )
+                if (emotion != null) {
+                    Text(
+                        text = "#$emotion",
+                        color = ReedTheme.colors.contentBrand,
+                        style = ReedTheme.typography.label1SemiBold,
+                    )
+                }
                 Text(
                     text = createdAt,
                     color = ReedTheme.colors.contentTertiary,
                     style = ReedTheme.typography.caption1Regular,
                 )
             }

• 필요 시, emotion == null인 경우에 대한 플레이스홀더(예: 기본 아이콘/태그 숨김) 정책을 디자인과 합의해 주세요.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/BookUpdateBottomSheet.kt (1)

31-31: '14.dp' 직접값 사용 — 디자인 토큰(또는 공통 상수)로 추출을 권장합니다

이번 PR 전반에서 vertical padding을 14.dp로 표준화하고 있어 유지보수 용이성을 위해 토큰화가 좋습니다. 우선 파일 로컬 상수로 감싸도 효과가 큽니다. (동일 패턴이 BookRegisterBottomSheet.kt에도 존재)

적용 예시:

-            .padding(vertical = 14.dp),
+            .padding(vertical = VerticalPaddingMd),

파일 상단(또는 하단)에 다음 상수 추가:

// 파일 상단에 추가
private val VerticalPaddingMd = 14.dp

장기적으로는 ReedTheme.spacing 등에 정식 토큰(예: interactiveVerticalMd)을 추가해 전체 코드베이스에서 일관되게 사용할 수 있도록 하는 것을 추천합니다.

Also applies to: 144-144

feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/book/component/BookRegisterBottomSheet.kt (1)

31-31: 중복되는 '14.dp' 매직넘버 — 공통 상수 또는 디자인 토큰으로 치환하면 좋습니다

BookUpdateBottomSheet와 동일한 14.dp vertical padding이 반복됩니다. 공통 상수로 감싸거나 디자인 토큰으로 승격하면 이후 스펙 변경 시 수정 범위를 줄일 수 있습니다.

적용 예시:

-            .padding(vertical = 14.dp),
+            .padding(vertical = VerticalPaddingMd),

파일 상단(또는 하단)에 다음 상수 추가:

private val VerticalPaddingMd = 14.dp

전사적으로는 ReedTheme.spacing에 정식 토큰을 만드는 방향을 추천드립니다.

Also applies to: 143-143

feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/book/component/BookRegisterSuccessBottomSheet.kt (1)

54-61: 여백 조정은 적합. 상단 Spacer 추가 의도 확인 + 이미지 접근성(contentDescription) 재검토 제안

• 상단 패딩(top spacing5) 뒤에 Spacer(spacing3)를 추가하고, 이미지 하단 Spacer를 spacing5로 키운 변경은 시각적 리듬 측면에서 타당해 보입니다. 다만 상단 패딩과 Spacer가 누적되어 과한 여백이 되지 않는지 의도 확인 부탁드립니다.
• 이미지가 장식용이라면 스크린 리더 노이즈를 줄이기 위해 contentDescription = null을 권장합니다. 의미 전달이 필요하다면 stringResource로 관리해 주세요.

적용 예시:

 Image(
     painter = painterResource(R.drawable.img_book_register_complete),
-    contentDescription = "Book Register Complete Image",
+    contentDescription = null,
     modifier = Modifier.height(120.dp),
 )

feature/splash/src/main/kotlin/com/ninecraft/booket/splash/SplashUi.kt (1)

70-74: textAlign을 효과적으로 적용하려면 가로 제약을 부여하세요

현재 Text는 기본적으로 wrap content라 textAlign = Center가 실질적으로 영향이 없을 수 있습니다(특히 2줄 이상일 때). 가로 제약을 주기 위해 fillMaxWidth를 추가하는 것을 권장합니다.

 Text(
     text = stringResource(R.string.splash_title),
+    modifier = Modifier.fillMaxWidth(),
     color = ReedTheme.colors.contentInverse,
     textAlign = TextAlign.Center,
     style = ReedTheme.typography.heading2SemiBold,
 )

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/component/HomeBanner.kt (1)

76-76: 수평 간격에는 width 사용을 권장

Row 내부 수평 간격이라면 의도를 명확히 하기 위해 size보다 width가 가독성에 유리합니다. 동작상 문제는 없으므로 선택적 변경 제안입니다.

-Spacer(modifier = Modifier.size(ReedTheme.spacing.spacing1))
+Spacer(modifier = Modifier.width(ReedTheme.spacing.spacing1))

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordSortBottomSheet.kt (1)

97-103: Divider를 컨텐츠 패딩에 맞춰 인셋하는 것을 제안

현재 Divider가 시트 전체 폭으로 그려져 헤더와 리스트의 좌우 여백과 불일치합니다. 컨텐츠 라인에 맞춘 인셋 Divider가 더 자연스러울 수 있습니다. 디자인이 풀블리드 Divider를 요구하지 않는다면 아래처럼 수평 패딩을 맞춰 주세요.

-                    if (index < recordSortItems.lastIndex) {
-                        HorizontalDivider(
-                            modifier = Modifier.fillMaxWidth(),
-                            thickness = 1.dp,
-                            color = ReedTheme.colors.dividerSm,
-                        )
-                    }
+                    if (index < recordSortItems.lastIndex) {
+                        HorizontalDivider(
+                            modifier = Modifier
+                                .fillMaxWidth()
+                                .padding(horizontal = ReedTheme.spacing.spacing5),
+                            thickness = 1.dp,
+                            color = ReedTheme.colors.dividerSm,
+                        )
+                    }

feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/ocr/OcrUi.kt (1)

316-321: 리스트 상단 여백은 item 대신 contentPadding으로 처리하면 더 간결합니다

불필요한 가상 아이템을 줄여 성능·접근성 측면에서 유리하고, 이후 헤더/푸터 추가 시에도 일관성이 좋아집니다.

아래처럼 변경을 제안드립니다.

             LazyColumn(
                 modifier = Modifier
                     .weight(1f)
-                    .padding(horizontal = ReedTheme.spacing.spacing5),
+                    .padding(horizontal = ReedTheme.spacing.spacing5),
+                contentPadding = PaddingValues(top = ReedTheme.spacing.spacing1),
                 verticalArrangement = Arrangement.spacedBy(ReedTheme.spacing.spacing2),
             ) {
-                item {
-                    Spacer(modifier = Modifier.height(ReedTheme.spacing.spacing1))
-                }
 
                 items(state.sentenceList.size) { index ->

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomeUi.kt (1)

193-193: Preview를 Success로 고정한 의도 이해됨 — 샘플 데이터 추가를 고민해보세요

현재는 빈 리스트일 경우 Empty 상태만 보여질 수 있습니다. 미리보기에서 실제 콘텐츠 카드/페이저 동작을 확인하려면 더미 recentBooks를 1~2개 넣는 것도 방법입니다.

feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/LibraryUi.kt (1)

170-182: TextAlign.Center만으로는 효과가 제한적 — fillMaxWidth 추가를 권장합니다

현재 Column이 wrap-content 폭이라 Text의 textAlign이 실제 화면 너비 기준 중앙 정렬로 보장되지 않습니다. 각 Text에 modifier.fillMaxWidth()를 추가하면 의도한 중앙 정렬이 확실해집니다.

             Text(
                 text = stringResource(R.string.library_empty_book_title),
                 color = ReedTheme.colors.contentPrimary,
+                modifier = Modifier.fillMaxWidth(),
                 textAlign = TextAlign.Center,
                 style = ReedTheme.typography.headline1SemiBold,
             )
             Spacer(modifier = Modifier.height(ReedTheme.spacing.spacing2))
             Text(
                 text = stringResource(R.string.library_empty_book_description),
                 color = ReedTheme.colors.contentSecondary,
+                modifier = Modifier.fillMaxWidth(),
                 textAlign = TextAlign.Center,
                 style = ReedTheme.typography.body1Medium,
             )

feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/book/component/BookItem.kt (2)

27-27: em 단위 사용 시 import 추가 필요

아래 제안(문자 간격 em 활용)을 적용하려면 em import가 필요합니다.

 import androidx.compose.ui.unit.dp
-import androidx.compose.ui.unit.sp
+import androidx.compose.ui.unit.sp
+import androidx.compose.ui.unit.em

---

99-103: 문자 간격은 em, lineHeight는 기반 스타일에서 파생값으로 계산 권장

하드코딩된 16.sp을 기준으로 곱을 적용하면, 디자인 토큰(typography) 변경 시 여기만 값이 어긋납니다. 기반 스타일(fontSize)에서 파생시키고, letterSpacing은 em 단위를 권장합니다.

-                maxLines = 2,
-                style = ReedTheme.typography.body1SemiBold.copy(
-                    lineHeight = 16.sp * 1.4f,
-                    letterSpacing = 16.sp * 0.01f,
-                ),
+                maxLines = 2,
+                style = ReedTheme.typography.body1SemiBold.copy(
+                    lineHeight = ReedTheme.typography.body1SemiBold.fontSize * 1.4f,
+                    letterSpacing = 0.01.em,
+                ),

참고: letterSpacing의 em 사용은 Compose 버전에 따라 차이가 있을 수 있어, 팀이 사용하는 Compose 버전 호환 여부를 한 번만 확인 부탁드립니다.

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/component/BookCard.kt (2)

161-162: Seed 카운트 영역의 세로 패딩 9.dp 하드코딩 — 토큰화 여부 확인 요청

디자인 요구로 특정 수치(9.dp)가 필요한 경우라면 이해됩니다. 다만 동일 패턴이 반복된다면 spacing 토큰 추가를 고려해 주세요. 재사용성·일관성이 높아집니다.

---

227-229: EmptyBookCard의 테두리 토큰 전환 일관성 좋습니다 — BookCard 쪽도 동일 의도인지 확인 요청

EmptyBookCard는 width=ReedTheme.border.border1, color=borderPrimary로 갱신되었는데, 상단 BookCard의 외곽선(라인 71–74)은 여전히 1.dp + borderSecondary를 사용 중입니다. 의도된 차별화인지, 동일 정책 적용 누락인지 확인 부탁드립니다.

참고(예시):

.border(
    width = ReedTheme.border.border1,
    color = ReedTheme.colors.borderPrimary,
    shape = RoundedCornerShape(ReedTheme.radius.sm),
)

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt (2)

108-112: 프리뷰 데이터 보강 제안

Preview에서 빈 리스트를 전달하면 위 가드가 없을 때 크래시, 가드가 있어도 시각적으로 검증이 어려울 수 있습니다. 대표 값 하나를 넣어 실제 렌더링을 확인하는 편이 좋습니다.

-            emotionTags = persistentListOf(),
+            emotionTags = persistentListOf("따뜻함"),

---

64-64: 접근성 텍스트의 현지화 고려

contentDescription = "Emotion Graphic"은 영어 하드코딩입니다. 앱 언어에 맞게 문자열 리소스로 분리해 주세요. 장식용 이미지라면 null로 설정하는 것도 검토 가능합니다.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/record/component/ReviewBox.kt (2)

50-57: 중복 패턴 공용화 제안 (Avatar/Chip 컴포넌트로 추출)

RecordItem과 동일한 원형 이미지 + 배경 패턴이 반복됩니다. 크기만 파라미터화한 공용 컴포저블(예: EmotionAvatar(emotion, size))로 추출하면 유지보수성과 디자인 일관성이 좋아집니다.

---

51-53: 접근성 텍스트의 현지화 고려

여기도 contentDescription이 영어 하드코딩입니다. 문자열 리소스로 분리하거나 장식용일 경우 null 설정을 검토해 주세요.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt:29-37
Timestamp: 2025-07-31T23:17:40.054Z
Learning: Reed-Android 프로젝트에서는 API가 준비되지 않은 상황에서 UI를 먼저 구현하고, API 연동 시점에 하드코딩된 데이터를 실제 데이터로 교체하는 개발 방식을 사용한다. RecordItem 컴포넌트의 emotionTags 매개변수도 API 연동 시점에 `text = emotionTags.joinToString(separator = "·") { "#$it" }`로 적용될 예정이다.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#63
File: feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/LibraryPresenter.kt:28-53
Timestamp: 2025-07-22T05:19:10.071Z
Learning: feature/library의 LibraryPresenter에서 현재 FilterChipState의 카운트 값들은 UI 확인용 더미 데이터이며, API가 준비되면 실제 데이터로 교체될 예정입니다.